Skip to content

fix(richdocuments): Use filename from DownloadAs postmessage for correct file extension#16623

Open
Rash419 wants to merge 2 commits intonextcloud:masterfrom
Rash419:fix-downlaod-as
Open

fix(richdocuments): Use filename from DownloadAs postmessage for correct file extension#16623
Rash419 wants to merge 2 commits intonextcloud:masterfrom
Rash419:fix-downlaod-as

Conversation

@Rash419
Copy link

@Rash419 Rash419 commented Mar 3, 2026

🖼️ Screenshots

🏚️ Before 🏡 After
B A

🏁 Checklist

  • Tests written, or not not needed

…ect file extension

Signed-off-by: Rashesh Padia <rashesh.padia@collabora.com>

else -> {
val downloadFileName = downloadJson.optString(FILENAME, fileName)
downloadFile(url, downloadFileName)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, thank you for the PR.

I have a quick question regarding the fileName handling. Is the fileName always incorrect, or is it only missing the extension in some cases? Also, can we confirm whether downloadJson.optString(FILENAME, fileName) consistently returns valid data?

Could you please share the steps to reproduce the issue and how you tested this? That would help us better understand the scenario and verify the behavior on our side.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi,

fileName extension is incorrect when using "Download as" button to download file into different format. In Collabora Online side, I added CollaboraOnline/online#14344 filename property to the DownloadAs postmessage JSON which will filename with correct extension.

Steps to reproduce:

  1. Open writer document
  2. Hamburger Menu > Download as -> rtf
  3. File will have wrong extension on download

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants